Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add test for admin search #113

Merged
merged 2 commits into from
May 28, 2024

Conversation

PetrDlouhy
Copy link
Collaborator

@PetrDlouhy PetrDlouhy commented May 28, 2024

Test for #107

@PetrDlouhy PetrDlouhy force-pushed the test_admin_search branch from b3aa3a0 to ad2bd7a Compare May 28, 2024 12:27
@PetrDlouhy
Copy link
Collaborator Author

@adamcharnock I added a few more tests to try also filtering.
Should fail, if #107 was not fixed.

The tests are not passing, because master is broken.

@adamcharnock
Copy link
Owner

Whoops! I did not mean to add a number of those migrations, sorry about that. Checks are running now

@adamcharnock
Copy link
Owner

Ok, master should is fixed now. Merge as you wish.

@PetrDlouhy PetrDlouhy force-pushed the test_admin_search branch from ad2bd7a to 6862a51 Compare May 28, 2024 13:22
@PetrDlouhy PetrDlouhy merged commit b98b709 into adamcharnock:master May 28, 2024
17 checks passed
@PetrDlouhy
Copy link
Collaborator Author

@adamcharnock OK, Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants